Skip to content

[stable8] fix(NcPasswordField): reset validation on change#8357

Merged
susnux merged 2 commits intostable8from
backport/8350/stable8
Apr 20, 2026
Merged

[stable8] fix(NcPasswordField): reset validation on change#8357
susnux merged 2 commits intostable8from
backport/8350/stable8

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Mar 25, 2026

Backport of #8350

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Remove all the empty commits

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews feature: input-field Covering the InputField, TextField, ... labels Mar 25, 2026
@backportbot backportbot Bot added this to the 8.37.0 milestone Mar 25, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.85%. Comparing base (f57b50c) to head (49f10b5).
⚠️ Report is 53 commits behind head on stable8.

Additional details and impacted files
@@           Coverage Diff            @@
##           stable8    #8357   +/-   ##
========================================
  Coverage    45.85%   45.85%           
========================================
  Files          194      194           
  Lines         4892     4892           
  Branches      1262     1200   -62     
========================================
  Hits          2243     2243           
- Misses        2561     2563    +2     
+ Partials        88       86    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

- tests given payload length only

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- do not send a request on empty field

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the backport/8350/stable8 branch from 3f0b8d8 to 49f10b5 Compare March 25, 2026 16:57
Copy link
Copy Markdown
Contributor

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rewrote logic in NcPasswordField watcher to match main branch behaviour

@Antreesy Antreesy marked this pull request as ready for review March 25, 2026 16:59
@Antreesy Antreesy modified the milestones: 8.37.0, 8.38.0 Apr 15, 2026
@susnux susnux merged commit 2b88087 into stable8 Apr 20, 2026
21 checks passed
@susnux susnux deleted the backport/8350/stable8 branch April 20, 2026 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug Something isn't working feature: input-field Covering the InputField, TextField, ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants